Skip to content

fix: add register-user endpoint for Telegram bot tracking#651

Merged
ShaerWare merged 1 commit intomainfrom
fix/telegram-register-user
Mar 25, 2026
Merged

fix: add register-user endpoint for Telegram bot tracking#651
ShaerWare merged 1 commit intomainfrom
fix/telegram-register-user

Conversation

@ShaerWare
Copy link
Owner

Summary

  • Bug: UserTrackerMiddleware POSTs to /instances/{id}/register-user but endpoint didn't exist — users never recorded
  • Fix: Add POST /instances/{id}/register-user endpoint (no auth, internal bot→orchestrator call)
  • Fix: TelegramSessionService.register_user() — creates/updates user in telegram_sessions without requiring chat_session_id
  • Fix: chat_session_id now nullable in TelegramSession model (users register before first chat)
  • Migration: scripts/migrate_telegram_sessions_nullable.py for SQLite schema change

NEWS

👥 Пользователи Telegram-ботов теперь отслеживаются!

Исправили баг, из-за которого пользователи ботов не записывались в базу данных. Теперь каждый, кто напишет боту, появится в списке пользователей в админке.

Test plan

  • Write to any bot in Telegram
  • Check Users tab in admin → should show the user
  • Verify user has name, username, Telegram ID, dates
  • Multiple bots: users appear per-bot

🤖 Generated with Claude Code

- Add POST /instances/{id}/register-user endpoint (no auth, internal)
- Add TelegramSessionService.register_user() method
- Make chat_session_id nullable in TelegramSession model
- Migration script for SQLite schema change

## NEWS

👥 **Пользователи Telegram-ботов теперь отслеживаются!**

Исправили баг, из-за которого пользователи ботов не записывались
в базу данных. Теперь каждый, кто напишет боту, появится в списке
пользователей в админке.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ShaerWare ShaerWare merged commit 9d4da84 into main Mar 25, 2026
3 checks passed
@ShaerWare ShaerWare deleted the fix/telegram-register-user branch March 25, 2026 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant